Skip to content

Enhance exhibitor submission UX with loading feedback and status visibility#29

Open
sandy191020 wants to merge 1 commit intofossasia:mainfrom
sandy191020:enhance-exhibitor-submission-ux
Open

Enhance exhibitor submission UX with loading feedback and status visibility#29
sandy191020 wants to merge 1 commit intofossasia:mainfrom
sandy191020:enhance-exhibitor-submission-ux

Conversation

@sandy191020
Copy link

Fixes #

Overview

This PR improves the exhibitor submission workflow by enhancing user feedback and status visibility.

Changes Made

UI/UX Improvements

  • Added loading indicator during form submission
  • Disabled submit button to prevent duplicate submissions
  • Displayed success confirmation message after submission
  • Added "Pending Approval" status badge in exhibitor UI

Backend Enhancement

  • Introduced a persistent status field for exhibitors
  • Default status set to "Pending Approval"
  • Enables consistent state across reloads and API usage

Error Handling

  • Improved user-friendly error messages for failed submissions
  • Prevented raw error outputs in UI

User Experience

After these changes:

  • Users see a loading spinner while submitting
  • Duplicate submissions are prevented
  • Clear success feedback is shown
  • Submitted exhibitors display a "Pending Approval" status

Notes

  • While the original issue focused on UI feedback, a minimal backend enhancement was added to persist the status and improve consistency.
  • I’m happy to refine or scope this further based on maintainer feedback.

Testing

  • Verified functionality through manual testing
  • Added basic test assertion for default "pending" status
  • Full test suite could not be executed due to local environment limitations

Impact

This improves clarity, prevents user confusion, and enhances the overall exhibitor submission experience.

Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry @sandy191020, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

@sandy191020
Copy link
Author

Fixes #28

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant